Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update controller-runtime, Kubernetes and kyaml dependencies #518

Merged
merged 3 commits into from
May 25, 2023

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented May 24, 2023

This deals with various breaking changes in controller-runtime, as
documented in the release notes:
https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.15.0

In short:

  • Watches now use a client.Object instead of a source.Kind.
  • handler.MapFunc signature accepts a Go context, which is used to
    log any errors, instead of silently ignoring them and/or panicking.
  • Fake clients used in tests are now configured using
    WithStatusSubresource to enable the correct behavior for status
    updates and patches.
  • Max concurrent reconciles and panic recovery is configured on the
    manager, instead of configuring them per reconciler instance.
  • Various manager configuration options have been moved to new
    structures and/or fields.

In addition to this, all other dependencies which had updates are
updated to their latest (compatible) versions as well. Including an
update of kyaml to match Kustomize v5.

Part of: fluxcd/flux2#3907 and fluxcd/flux2#3564

- github.com/fluxcd/pkg/apis/meta to version 1.1.0
- github.com/fluxcd/source-controller/api to version 1.0.0-rc.3
- k8s.io/apimachinery to version 0.27.2
- sigs.k8s.io/controller-runtime to version 0.15.0

Signed-off-by: Hidde Beydals <hidde@hhh.computer>
@hiddeco hiddeco added the dependencies Pull requests that update a dependency label May 24, 2023
@hiddeco hiddeco force-pushed the update-ctrl-runtime branch 8 times, most recently from 19bcbf1 to 4d9d5e6 Compare May 25, 2023 07:55
@stefanprodan
Copy link
Member

Can you please mention kyaml as this is the major update, controller-runtime has no affect on end-users.

@hiddeco hiddeco force-pushed the update-ctrl-runtime branch 2 times, most recently from 9720e24 to c498a58 Compare May 25, 2023 07:59
@hiddeco hiddeco changed the title Update controller-runtime and Kubernetes dependencies Update controller-runtime, Kubernetes and kyaml dependencies May 25, 2023
hiddeco added 2 commits May 25, 2023 10:08
This deals with various breaking changes in controller-runtime, as
documented in the release notes:
https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.15.0

In short:

- `Watches` now use a `client.Object` instead of a `source.Kind`.
- `handler.MapFunc` signature accepts a Go context, which is used to
  log any errors, instead of silently ignoring them and/or panicking.
- Fake clients used in tests are now configured using
  `WithStatusSubresource` to enable the correct behavior for status
  updates and patches.
- Max concurrent reconciles and panic recovery is configured on the
  manager, instead of configuring them per reconciler instance.
- Various manager configuration options have been moved to new
  structures and/or fields.

In addition to this, all other dependencies which had updates are
updated to their latest (compatible) versions as well. Including an
update of kyaml to match Kustomize v5.

Signed-off-by: Hidde Beydals <hidde@hhh.computer>
Signed-off-by: Hidde Beydals <hidde@hhh.computer>
@hiddeco hiddeco force-pushed the update-ctrl-runtime branch from c8dc943 to 7cc9c9c Compare May 25, 2023 08:08
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks @hiddeco 🏅

@hiddeco hiddeco merged commit 4ac647a into main May 25, 2023
@hiddeco hiddeco deleted the update-ctrl-runtime branch May 25, 2023 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants